Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Allow date selection for Entry, Minor adjustments #32

Merged
merged 13 commits into from
Jun 3, 2024
Merged

Allow date selection for Entry, Minor adjustments #32

merged 13 commits into from
Jun 3, 2024

Conversation

therazix
Copy link
Owner

@therazix therazix commented Jun 3, 2024 •
edited
Loading

  • c02ebf3 - Na některých zařízení byl text bílý a nešel vidět.
  • 83dfa8b - Z textu nebylo jasné, k čemu TimeMachine je.
  • f12a632 - Overflow byl už opraven v 5ff7894. Tohle zbylo po rebasu a akorát to dává hvězdičku úplně dolů.
  • dc52416 - Specifikuje barvu textu v kalendáři. SelectedTodayTextColor ale z nějakého důvodu nefunguje a furt je to černý, což na modrým pozadí není moc vidět.
  • 0f3e06c - Odstraňuje EditedAt u Entry a nechává jen jedno datum, které lze i měnit. Při vytváření Entry lze také předvyplnit datum tím způsobem, že se klikne na konkrétní den v kalendáři. Notifikace by měly také fungovat. Je nutné odstranit data aplikace, aby se vytvořila nová databáze, jinak atributy u Entry nebudou aktuální.
  • 12f6d31 - Přidává labely do grafů, aby bylo lépe poznat, co za datum je aktuálně vybráno. Také upravuje tlačítka a přidává loading spinner při delším načítání.
  • 74f34a0 - Sjednocuje způsob nastavení Titlů v AppShellu.
  • a45bdcd - Přidává krátký text na vysvětlení.
  • b367b59 - Na některých zařízení měl slider bílou čáru, která nešla vidět.
  • 4dcb7bb - TimePicker na některých zařízení nešel vůbec vidět. Zajímavé je, že DatePicker to nedělá.
  • 41cfd4b - Při velkém počtu labels se CollectionView furt rozšiřovalo, což zbytečne roztahovalo stránky. Část filter popupu potom kvůli tomu nebyla vidět.
  • e6329cb - Po kliknutí na pin se přešlo na detail záznamu, ale tlačítko zpět vracelo na list view. Teď už to správně vrací do mapy.

@therazix therazix requested a review from yoshi29 June 3, 2024 10:26
yoshi29
yoshi29 approved these changes Jun 3, 2024
View reviewed changes
Copy link
Collaborator

@yoshi29 yoshi29 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Skvěle popsané commity 🙂

@yoshi29 yoshi29 merged commit 9f98a34 into main Jun 3, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yoshi29 yoshi29 yoshi29 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@therazix @yoshi29